Skip to content

docs: add pre-commit linting rule for TypeScript changes to CLAUDE.md#3699

Merged
zackverham merged 1 commit intomainfrom
add-claude-linting-suggestions
Mar 11, 2026
Merged

docs: add pre-commit linting rule for TypeScript changes to CLAUDE.md#3699
zackverham merged 1 commit intomainfrom
add-claude-linting-suggestions

Conversation

@zackverham
Copy link
Collaborator

@zackverham zackverham commented Mar 11, 2026

Summary

  • Adds a "Pre-Commit Checks" section to CLAUDE.md instructing Claude to always run TypeScript linting (just lint from extensions/vscode/) before committing or pushing TypeScript changes.

Intent

Ensures Claude Code catches linting errors before they make it into commits, reducing CI failures and review churn on TypeScript changes.

Type of Change

  • Documentation

Approach

Added a new section between "Git Workflow" and "CHANGELOG Conventions" in the root CLAUDE.md, keeping it alongside the existing git workflow rules where it's most relevant.

User Impact

No user-facing impact. This only affects Claude Code's behavior when making TypeScript changes in the repository.

Automated Tests

N/A — documentation-only change.

Directions for Reviewers

Verify the new "Pre-Commit Checks" section in CLAUDE.md is clear and placed appropriately.

Checklist

  • I have updated the root CHANGELOG.md to cover notable changes.

🤖 Generated with Claude Code

Adds a Pre-Commit Checks section instructing to always run `just lint`
from `extensions/vscode/` before committing or pushing TypeScript changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@zackverham zackverham marked this pull request as ready for review March 11, 2026 19:57
@zackverham zackverham requested a review from a team as a code owner March 11, 2026 19:57
@zackverham zackverham merged commit fb72372 into main Mar 11, 2026
14 checks passed
@zackverham zackverham deleted the add-claude-linting-suggestions branch March 11, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants